Skip to content

feat(skill sandbox): tip token support - #891

Merged
warm-wm merged 3 commits into
volcengine:mainfrom
CatherineKu:feat/skill-sandbox-tip-token-support
Aug 21, 2026
Merged

feat(skill sandbox): tip token support#891
warm-wm merged 3 commits into
volcengine:mainfrom
CatherineKu:feat/skill-sandbox-tip-token-support

Conversation

@CatherineKu

Copy link
Copy Markdown
Contributor

Still in test

@CatherineKu
CatherineKu marked this pull request as draft August 14, 2026 03:07
@CatherineKu CatherineKu changed the title WIP(Skill sandbox): tip token support feat(skill sandbox): tip token support Aug 14, 2026
@CatherineKu
CatherineKu force-pushed the feat/skill-sandbox-tip-token-support branch 3 times, most recently from bdce51f to 2d0e4f7 Compare August 19, 2026 05:35
@CatherineKu
CatherineKu force-pushed the feat/skill-sandbox-tip-token-support branch from fb90078 to 67a9d3d Compare August 19, 2026 13:37
@CatherineKu
CatherineKu marked this pull request as ready for review August 19, 2026 13:39
@CatherineKu

Copy link
Copy Markdown
Contributor Author

Add inbound_auth passthrough for Skills Sandbox.

execute_skills now reads the AgentKit SDK captured inbound_auth credential
and forwards it as an inbound_auth header when calling the Skills Sandbox.
This lets sandbox agents and downstream MCP tools consume the original inbound
auth credential without additional token exchange.
This also adds redacted debug logging before sandbox dispatch, reporting only
presence, length, and sha256 digest, plus focused coverage for inbound_auth
passthrough and empty credential behavior.

Validation:
Ran targeted Skills Sandbox tests with agentkit-sdk-python==0.8.3 and
a2a-sdk==0.3.15.

9 sample contract tests passed.
End to End User -- Runtime -- Skill Sandbox -- MCP tool test passed.

@warm-wm
warm-wm merged commit f06cbdd into volcengine:main Aug 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants